Skip to content

Stabilize the stdarch_neon_dotprod feature#2120

Open
lupodevelop wants to merge 1 commit into
rust-lang:mainfrom
lupodevelop:stabilize-neon-dotprod
Open

Stabilize the stdarch_neon_dotprod feature#2120
lupodevelop wants to merge 1 commit into
rust-lang:mainfrom
lupodevelop:stabilize-neon-dotprod

Conversation

@lupodevelop
Copy link
Copy Markdown

Stabilizes the NEON dot product intrinsics (vdot*), tracking issue rust-lang/rust#117224.

Gated behind stdarch_neon_dotprod since 2023. The dotprod target feature and is_aarch64_feature_detected!("dotprod") are already stable, so this is an attribute-only change, same pattern as the fp16 stabilization in 0f9516b.

The vdot*_laneq* blocks had a plain unstable with no arch split.
I gave them the not(arm) split like the others for consistency.

Regenerated with stdarch-gen-arm, the diff is the stability attribute only.

Needs an FCP.

Stabilizes the NEON dot product intrinsics tracked by
rust-lang/rust#117224:

  vdot{q}_{s,u}32
  vdot{q}_lane_{s,u}32
  vdot{q}_laneq_{s,u}32

These have been available unstably since 2023. The `dotprod` target
feature and `is_aarch64_feature_detected!("dotprod")` are already
stable, so this only flips the intrinsics' stability attribute on
aarch64 and arm64ec.

On AArch32 the intrinsics remain unstable under
`stdarch_arm_neon_intrinsics` (rust-lang/rust#111800), consistent with
the rest of the AArch32 NEON surface.
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 18, 2026

r? @folkertdev

rustbot has assigned @folkertdev.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @Amanieu, @adamgemmell, @davidtwco, @folkertdev, @sayantn
  • @Amanieu, @adamgemmell, @davidtwco, @folkertdev, @sayantn expanded to Amanieu, adamgemmell, davidtwco, folkertdev, sayantn
  • Random selection from Amanieu, adamgemmell, davidtwco, folkertdev, sayantn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants